any update or review on this PR? if you need more detail i can provide.

@MorrisJobke any comment ?

maybe someone from @owncloud/javascript want to have a look :)

@patelparas you can try to ask people who contributed to this app in the past to review

@kazsw @kralizeck @mirekys @adobo @pitbulk @sigmunau

The code looks fine, but I don't have an environment to test this

Actually i have developed and tested. and deployed SAML-ADFS in my Production in Feb-2016. it is working fine we didn't find any issue yet...... but again you need to check with latest version......

and user_saml app need other changes also like.....
e.g. in case of user use ObjectGUID as uid a need to decode GUID like this:

public static function base64ToobjectGUID($guid_b64) {
            $guid_bin = base64_decode($guid_b64);
            return join('-', array(
                bin2hex(strrev(substr($guid_bin, 0, 4))),
                bin2hex(strrev(substr($guid_bin, 4, 2))),
                bin2hex(strrev(substr($guid_bin, 6, 2))),
                bin2hex(substr($guid_bin, 8, 2)),
                bin2hex(substr($guid_bin, 10, 6))
            ));
 }

I think we need to rewrite SimplePHPSAML deployment and integration manual step by step. existing one is not sufficient. i know individual contributor will not take interest in this contribution because ADFS-SSO is need for Enterprise Use Case only.

One question is whether this change would be unacceptable for other SAML user, or compatible ?
If not, then maybe this needs a config switch where this is disabled by default.